Skip to content

fix(commands): transfer abs_fnames back after /agent temp-coder turn#568

Merged
dwash96 merged 1 commit into
cecli-dev:v0.100.6from
Digital-Defiance:pr/agent-file-transfer
Jun 12, 2026
Merged

fix(commands): transfer abs_fnames back after /agent temp-coder turn#568
dwash96 merged 1 commit into
cecli-dev:v0.100.6from
Digital-Defiance:pr/agent-file-transfer

Conversation

@JessicaMulein

Copy link
Copy Markdown

Summary

_generic_chat_command creates a temporary coder for /agent (and other slash commands with args). Files added by ContextManager during the turn were on the temp coder and got discarded when SwitchCoderSignal switched back to the original.

Now transfers abs_fnames and abs_read_only_fnames from the temp coder back to the original before raising SwitchCoderSignal. This ensures files added during /agent turns persist in the session and are visible to subsequent GET /sessions/{id} (files_in_chat).

Test plan

  • python -m pytest tests/ -x -q (414 passed)
  • Verified via BrightVision session debug export: files_in_chat now reflects ContextManager-added files after /agent turn completes

_generic_chat_command creates a temporary coder for /agent (and other
slash commands with args). Files added by ContextManager during the
turn were on the temp coder and got discarded when SwitchCoderSignal
switched back to the original.

Now transfer abs_fnames and abs_read_only_fnames from the temp coder
back to the original before raising SwitchCoderSignal. This ensures
files added during /agent turns persist in the session and are visible
to subsequent GET /sessions/{id} (files_in_chat).
@dwash96 dwash96 changed the base branch from main to v0.100.6 June 12, 2026 19:02
@dwash96 dwash96 merged commit 4e5ac47 into cecli-dev:v0.100.6 Jun 12, 2026
12 checks passed
@dwash96 dwash96 mentioned this pull request Jun 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants